Skip to content

Update CICD to move label deletion to Jenkinsfile #2568

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

BruceKropp-Raytheon
Copy link
Collaborator

@BruceKropp-Raytheon BruceKropp-Raytheon commented Jan 24, 2025

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
  • Commit 'test_changes.list' from previous step

Description:

API call to github from code requires a credential token, which is only known within Jenkinsfile.
So moving the API call that removes a test label out of the post script and into Jenkinsfile.

Commit Message:

Update CICD to move label deletion to Jenkinsfile

* UFSWM - CI only

Priority:

  • Normal

Git Tracking

  • None

Sub component Pull Requests:

  • None

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • GaeaC5
    • GaeaC6
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

Signed-off-by: Bruce Kropp <bruce.kropp@raytheon.com>
@jkbk2004
Copy link
Collaborator

@BruceKropp-Raytheon we will combine this pr to merge with #2548.

FernandoAndrade-NOAA pushed a commit that referenced this pull request Feb 3, 2025
…temperature and radiative flux biases + Allow model_configure to provide a path for FV3ATM output #2581 + Update CICD to move label deletion to Jenkinsfile #2568 (#2548)

* UFSWM - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases 
    + Allow users to provide custom paths for fv3atm output. Default `./` is retained for the regression testing of the ufs-weather-model
    + Move CI label deletion to Jenkinsfile
  * FV3 - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases
    * ccpp-physics - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases
@FernandoAndrade-NOAA
Copy link
Collaborator

Updates merged with #2548, closing.

gspetro-NOAA pushed a commit to gspetro-NOAA/ufs-weather-model that referenced this pull request Feb 5, 2025
…temperature and radiative flux biases + Allow model_configure to provide a path for FV3ATM output ufs-community#2581 + Update CICD to move label deletion to Jenkinsfile ufs-community#2568 (ufs-community#2548)

* UFSWM - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases 
    + Allow users to provide custom paths for fv3atm output. Default `./` is retained for the regression testing of the ufs-weather-model
    + Move CI label deletion to Jenkinsfile
  * FV3 - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases
    * ccpp-physics - Updates to radiation, cloud, and land processes for improved surface temperature and radiative flux biases
@BruceKropp-Raytheon BruceKropp-Raytheon deleted the feature/fix_post_label branch February 5, 2025 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants